-
-
Notifications
You must be signed in to change notification settings - Fork 79
[18.0][MIG] github_connector: Migration to version 18.0 #132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…hod. [FIX] Add the organization_id.github_login to the github url Making the github url the form: https://github.com/OCA/account-analytic/tree/9.0 and not the https://github.com/account-analytic/tree/9.0 before this commit [IMP] Make travis happy [IMP] Fix error with teams sync We already have the organization in the team data, and github does not provide anymore in the result for the team data.
[IMP] raise error instead of logging in case of duplicates To avoid creating additional duplicates.
[FIX] add ondelete cascade [FIX] team creation from odoo [FIX] possibility to recursively load an existing object
[REF] github_connector: Fix typo "should not => does not" (OCA#6)
[FIX] github_connector: remove duplicated group
* Fixed JSON response parsing issue while fetching data by URL
* [IMP] Add try-except on branch analyze action Add try-except statements on analyze actions that could potentially crash the analyze process. * [REF] Fix error logging Changes include: - fix error values - use error instead of warning * [IMP] Fix error in analyze call Fix a programming mistake in the analyze call. By @RoelAdriaans * [IMP] Bump version number of github_connector Bump the version number of github_connector to '11.0.1.1.1'. * [IMP] Bump version number of github_connector Bump the version number of github_connector_odoo to '11.0.1.1.1'.
To play nice with Sentry.
[FIX] github_connector: keyerror 'blog' Description of the issue/feature this PR addresses: On new database error when syncing new github organization (example: OCA). Error traceback: KeyError: 'blog' Current behavior before PR: Synchronization gets blog key on `get_odoo_data_from_github` but blog field does not exist. When it tries to match the key with model attribute we get key error. Desired behavior after PR is merged: Get blog key from github data and show it on form view.
If source path does not exists before adding it to modules path, styles on website are corrupted, making impossible load them from assets.
Don't mix bytes and str
There's a mix between self and branch arguments in analyze methods.
…functional This field allows to define where addons are located, but it's hidden right now, because not everything is properly adjusted for paths different from root. Making this working, there's no need to hide the field.
linter Linter Linter default value to empty string in get()'s secure the code improvment of the code coding style coding style
Using os.command gets hung when there's a problem deleting the folder (for example, when no permissions), so this stucks the cron method or the manual process of updating source code. We avoid it using the Python utility for that through shutil. We also avoid an exception in that case enclosing the sentence in a try/except clause.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: interface-github-17.0/interface-github-17.0-github_connector Translate-URL: https://translation.odoo-community.org/projects/interface-github-17-0/interface-github-17-0-github_connector/
Currently translated at 100.0% (197 of 197 strings) Translation: interface-github-17.0/interface-github-17.0-github_connector Translate-URL: https://translation.odoo-community.org/projects/interface-github-17-0/interface-github-17-0-github_connector/it/
ef6b7c3 to
c76ec06
Compare
|
/ocabot migration github_connector |
c7dc633 to
894e7c0
Compare
894e7c0 to
ee2b805
Compare
victoralmau
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review OK
ee2b805 to
2ad62a1
Compare
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at b4aeb45. Thanks a lot for contributing to OCA. ❤️ |
TT56041
@Tecnativa @pedrobaeza @victoralmau could you please review this?
Supersedes #131